Skip to content

[feat] create setup logger#8

Draft
Joel-leal wants to merge 3 commits intomainfrom
joel/setup-logger
Draft

[feat] create setup logger#8
Joel-leal wants to merge 3 commits intomainfrom
joel/setup-logger

Conversation

@Joel-leal
Copy link

@Joel-leal Joel-leal commented Jan 19, 2026

##Descriptiom

A custom logger was implemented to standardize and structure application logs, improving traceability and observability across the system.

##Changes

[feat]: Add log examples with different criticalities.
[feat]: Created setup logger
[feat]: Block console.log

##Preview

Captura de Tela 2026-01-30 às 15 31 37

##Close #5

@Joel-leal Joel-leal marked this pull request as draft January 19, 2026 14:32
@Joel-leal Joel-leal requested a review from frattezi January 30, 2026 18:34
Copy link
Contributor

@frattezi frattezi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, just I consider unnecessary the custom log method, also, we want to enforce the correct usage of log levels (info, warn, error, debug) as it semantically makes more sense than all logging being a customLogger

@frattezi
Copy link
Contributor

frattezi commented Feb 7, 2026

Do you have your setup correctly set? As I open the files I see a lot of errors:

Screenshot 2026-02-07 at 13 39 41 (3)

Why there is still this custom log, that don't event exist on the class anymore, and why is it being used as a parameter inside the log? Not accepted btw.

Are you correctly running the server and checking logs as you develop? As I run the APP I see errors:

Screenshot 2026-02-07 at 13 44 31

For this you need to properly import the logger module inside video module

The logger module should be like this, I don't know what the other methods you are adding are doing. with this we already have a extendable log setup with all the main log types:

Screenshot 2026-02-07 at 13 46 39

Once you have the logger set you can simply:
Screenshot 2026-02-07 at 13 40 16

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Setup Logger

2 participants